fix promotions behavior and texts - #82
Conversation
|
Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖 Please select which version do you want to release:
And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.
|
|
Beep boop 🤖 I noticed you didn't make any changes at the
In order to keep track, I'll create an issue if you decide now is not a good time
|
|
|
||
| ### Changed | ||
|
|
||
| - Updated "U. Mult." label to "Multiplicador de Unidade" (PT), "Unit Multiplier" (EN), and "Multiplicador de Unidad" (ES) for better readability |
There was a problem hiding this comment.
Isso aqui foi alinhado com o time de localization
There was a problem hiding this comment.
@ataideverton Vou alinhar com eles. Vou separar em outro PR tambem para simplificar.
| listPrice: item.listPrice * 100, | ||
| price: item.price * 100, | ||
| sellingPrice: item.price * 100, | ||
| sellingPrice: item.sellingPrice * 100, |
There was a problem hiding this comment.
Não tem chance do selling price vir null?
There was a problem hiding this comment.
vou conferir
2d58ba5 to
fb3167e
Compare
|
Your PR has been merged! App is being published. 🚀 After the publishing process has been completed (check #vtex-io-releases) and doing A/B tests with the new version, you can deploy your release by running:
After that your app will be updated on all accounts. For more information on the deployment process check the docs. 📖 |
2 participants
What problem is this solving?
Fixes critical edge cases in quote creation and improves UI readability:
How to test it?
Test promotional price preservation:
Test $0.00 promotional price:
Test large currency values:
Test column labels:
Screenshots or example usage:
Screen.Recording.2026-01-16.at.13.14.34.mov
Related to / Depends on
Edge case reported in production with promotional pricing